Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Lower Memory Consumption while Evaluating Stratifiers #1731

Merged
merged 1 commit into from
May 19, 2024
Merged

Lower Memory Consumption while Evaluating Stratifiers #1731

merged 1 commit into from
May 19, 2024

Conversation

alexanderkiel
Copy link
Member

Closes: #1723

@alexanderkiel alexanderkiel self-assigned this May 17, 2024
@alexanderkiel alexanderkiel added this to the v0.26.3 milestone May 17, 2024
@codecov Codecov
Copy link

codecov bot commented May 17, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 97.59615% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 96.14%. Comparing base (4d67189) to head (2b9ecd6).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1731   +/-   ##
========================================
  Coverage    96.13%   96.14%           
========================================
  Files          259      261    +2     
  Lines        16838    16893   +55     
  Branches       339      338    -1     
========================================
+ Hits         16188    16241   +53     
- Misses         311      314    +3     
+ Partials       339      338    -1
Files Coverage Δ
...-resource-store/src/blaze/db/resource_store/kv.clj 100.00% <100.00%> (ø)
modules/db/src/blaze/db/tx_log/local.clj 98.55% <100.00%> (ø)
.../interaction/src/blaze/interaction/transaction.clj 99.03% <100.00%> (ø)
modules/interaction/src/blaze/interaction/util.clj 100.00% <100.00%> (ø)
modules/luid/src/blaze/luid.clj 100.00% <100.00%> (ø)
modules/luid/src/blaze/luid/spec.clj 100.00% <100.00%> (ø)
modules/luid/src/blaze/luid_spec.clj 100.00% <100.00%> (+14.28%) ⬆️
.../src/blaze/fhir/operation/evaluate_measure/cql.clj 100.00% <100.00%> (+0.76%) ⬆️
.../fhir/operation/evaluate_measure/measure/group.clj 100.00% <100.00%> (ø)
.../operation/evaluate_measure/measure/population.clj 100.00% <100.00%> (ø)
... and 3 more

@alexanderkiel alexanderkiel force-pushed the 1723-lower-memory-stratifiers branch 2 times, most recently from 96726f5 to 7af8b8c Compare May 18, 2024 11:29
@alexanderkiel alexanderkiel force-pushed the 1723-lower-memory-stratifiers branch from 7af8b8c to 2b9ecd6 Compare May 19, 2024 07:50
@alexanderkiel alexanderkiel changed the title [WIP] Lower Memory Consumption while Evaluating Stratifiers Lower Memory Consumption while Evaluating Stratifiers May 19, 2024
@alexanderkiel alexanderkiel merged commit 19ed772 into develop May 19, 2024
155 checks passed
@alexanderkiel alexanderkiel deleted the 1723-lower-memory-stratifiers branch May 19, 2024 08:15
@alexanderkiel alexanderkiel modified the milestones: v0.26.3, v0.27.0 Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexanderkiel alexanderkiel

Labels
module:evaluate-measure module:luid performance Performance improvement
Projects
None yet
Milestone
v0.27.0
Development

Successfully merging this pull request may close these issues.
1 participant
@alexanderkiel